Skip to content

Conversation

@andy-stark-redis
Copy link
Contributor

Added docs about the silent install config file.

@andy-stark-redis andy-stark-redis added the documentation Improvements or additions to documentation label Jun 28, 2024
@andy-stark-redis andy-stark-redis requested review from a team and yaronp68 June 28, 2024 10:34
@andy-stark-redis andy-stark-redis self-assigned this Jun 28, 2024
use_existing_rdi = true
ssl = true
# Uncomment this section and remove the [rdi.database] section above
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if you want RDI to create the db you still need to provide the RDI password to use
not sure if you always have to specify use_exisiting_rdi=false

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added some text to say that you should set use_exisiting_rdi=false because I figure it won't do any harm. We can remove this in the future if it isn't necessary.

| `password` | Password for the cluster. |

#### `rdi.database.certificates`

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use this if RDI database is using TLS or mTLS

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. Also, I've commented these props out in the example TOML file (TLS/mTLS props are also commented out in the config.yaml example here: https://redis.io/docs/latest/integrate/redis-data-integration/ingest/data-pipelines/data-pipelines/#the-configyaml-file).

@andy-stark-redis andy-stark-redis merged commit 2e1c283 into main Jun 28, 2024
@andy-stark-redis andy-stark-redis deleted the DOC-3917-silent-install-file branch June 28, 2024 13:24
@andy-stark-redis andy-stark-redis restored the DOC-3917-silent-install-file branch June 28, 2024 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants